Skip to content

fix(swift-ios): explain unreachable environments in the new-thread picker - #8621

Merged
t3dotgg merged 7 commits into
pingdotgg:t3code/rebuild-mobile-app-swiftfrom
saphid:swiftui/pr-131-unreachable-environments
Sep 6, 2026
Merged

fix(swift-ios): explain unreachable environments in the new-thread picker#8621
t3dotgg merged 7 commits into
pingdotgg:t3code/rebuild-mobile-app-swiftfrom
saphid:swiftui/pr-131-unreachable-environments

Conversation

@saphid

@saphid saphid commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

The SwiftUI new-thread picker now explains which environments are unreachable and offers Retry. Cached projects remain selectable while offline, so new tasks can enter the durable queue. Disabled environments still reject new tasks.

A connection loss preserves the selected project and its draft key, including when another computer has the same repository. This corrects the earlier change that blocked offline projects and could reset a draft before its pending save completed.

Rebased onto t3code/rebuild-mobile-app-swift at 614a7b672a. Validation: 115 focused native tests passed on 7d2b1ae65885526f51f92040c23781022e2da896. The tests cover the connected-to-offline transition, stable project and draft identity, durable queuing, disabled environments, reachability messages, and Retry state. Range review and git diff --check passed.

Original visual evidence

These captures belong to earlier revisions. They were not recaptured for this update.

Original mixed catalog

Original no-project state

Original mixed-catalog interaction video

Tracking: native work item 131.

Modernized and fixed with GPT-6 in Codex.

@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 2140c4f7-e778-4e84-a3b3-4e31afd55b3d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:L 100-499 changed lines (additions + deletions). labels Aug 29, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9e6ff8e3fa55ff56a1f3eaab97826b9de34ebcbe. Configure here.

Comment thread apps/swift-ios/Features/Workspace/DailyUXModels.swift
@macroscopeapp

macroscopeapp Bot commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — The PR adds unreachable-environment UI and retry actions while changing existing new-task routing and offline project/queue behavior. These production-path behavior changes extend beyond a small isolated fix, so the scope merits human review.

Not approved because:

  • Monthly spending limit reached (workspace setting). Approvability relies on correctness review in order to determine eligibility

Review your spending limits in Billing settings. You can add or adjust custom eligibility rules. Learn more.

Comment thread apps/swift-ios/Features/Workspace/NewThreadView.swift
@github-actions github-actions Bot added size:XL 500-999 changed lines (additions + deletions). and removed size:L 100-499 changed lines (additions + deletions). labels Sep 4, 2026
@saphid saphid changed the title SwiftUI: explain enabled-but-unreachable environments in the new-thread picker fix(swift-ios): explain unreachable environments in the new-thread picker Sep 5, 2026
saphid and others added 7 commits September 6, 2026 15:13
Reconnecting is a transient state whose HTTP fallback still serves work,
so announcing those environments as unreachable in the picker, empty
state, and compose hint contradicted the sidebar and connection hub,
which present reconnecting separately from offline. Only disconnected
environments are unreachable now.

Co-Authored-By: Claude <noreply@anthropic.com>
@t3dotgg
t3dotgg force-pushed the swiftui/pr-131-unreachable-environments branch from 95d3da6 to 7d2b1ae Compare September 6, 2026 22:19
@t3dotgg
t3dotgg merged commit eab37a3 into pingdotgg:t3code/rebuild-mobile-app-swift Sep 6, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL 500-999 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants